Skip to content

feat: remove peak limit for optimization and adjust autoPeakPicking parameters#4241

Open
jobo322 wants to merge 6 commits into
mainfrom
remove-number-of-peaks-limit-to-optimize
Open

feat: remove peak limit for optimization and adjust autoPeakPicking parameters#4241
jobo322 wants to merge 6 commits into
mainfrom
remove-number-of-peaks-limit-to-optimize

Conversation

@jobo322

@jobo322 jobo322 commented Jun 30, 2026

Copy link
Copy Markdown
Member

No description provided.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 30, 2026

Copy link
Copy Markdown

Deploying nmrium with  Cloudflare Pages  Cloudflare Pages

Latest commit: 39b4dbe
Status: ✅  Deploy successful!
Preview URL: https://230c8c42.nmrium.pages.dev
Branch Preview URL: https://remove-number-of-peaks-limit.nmrium.pages.dev

View logs

@jobo322 jobo322 force-pushed the remove-number-of-peaks-limit-to-optimize branch from 5be4c43 to 51d58ba Compare July 2, 2026 14:32
@jobo322 jobo322 force-pushed the remove-number-of-peaks-limit-to-optimize branch from 51d58ba to 77518d7 Compare July 2, 2026 14:33
@lpatiny

lpatiny commented Jul 2, 2026

Copy link
Copy Markdown
Member

14.zip

Peak picking on this spectrum does not work at all.

image

Take care also that if we select the 50 more intense peak it should be the more intense in absolute value if both top / bottom are selected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants